Skip to content

feat(web-ongoing-operations): fix translate keys#23010

Open
louisbensiovh wants to merge 3 commits into
masterfrom
fix/DCE-218
Open

feat(web-ongoing-operations): fix translate keys#23010
louisbensiovh wants to merge 3 commits into
masterfrom
fix/DCE-218

Conversation

@louisbensiovh

Copy link
Copy Markdown
Contributor

Description

Modify the translate keys

Ticket Reference: #DCE-218

Copilot AI review requested due to automatic review settings July 9, 2026 08:21
@louisbensiovh
louisbensiovh requested a review from a team as a code owner July 9, 2026 08:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the i18n translation key pattern used by the “string argument” update UI in web-ongoing-operations, aligning labels with the domain_operations_update_* namespace (ticket DCE-218).

Changes:

  • Update UpdateStringComponent to use domain_operations_update_${argumentKey} instead of domain_operations_update_key_${argumentKey}.
  • Rename a subset of French (fr_FR) translation keys to the new pattern.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
packages/manager/apps/web-ongoing-operations/src/components/Update/Content/Update.String.component.tsx Switches the translation key format used for string-argument field labels.
packages/manager/apps/web-ongoing-operations/public/translations/dashboard/Messages_fr_FR.json Renames several domain_operations_update_key_* entries to domain_operations_update_* in French translations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

<FormField className="w-1/4" invalid={error != null && error.length > 0}>
<FormFieldLabel>
{t(`domain_operations_update_key_${argumentKey}`)}
{t(`domain_operations_update_${argumentKey}`)}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using translator fast pass to correct this

Copilot AI review requested due to automatic review settings July 9, 2026 08:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 9, 2026 08:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

<FormField className="w-1/4" invalid={error != null && error.length > 0}>
<FormFieldLabel>
{t(`domain_operations_update_key_${argumentKey}`)}
{t(`domain_operations_update_${argumentKey}`)}
Copilot AI review requested due to automatic review settings July 9, 2026 09:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

Copilot AI review requested due to automatic review settings July 9, 2026 15:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

<FormField className="w-1/4" invalid={error != null && error.length > 0}>
<FormFieldLabel>
{t(`domain_operations_update_key_${argumentKey}`)}
{t(`domain_operations_update_${argumentKey}`)}
Copilot AI review requested due to automatic review settings July 15, 2026 13:09
@louisbensiovh
louisbensiovh requested a review from a team as a code owner July 15, 2026 13:10
@github-actions github-actions Bot added the dependencies Pull requests that update a dependency file label Jul 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

<FormField className="w-1/4" invalid={error != null && error.length > 0}>
<FormFieldLabel>
{t(`domain_operations_update_key_${argumentKey}`)}
{t(`domain_operations_update_${argumentKey}`.toLowerCase())}
Comment on lines 65 to 67
<li key={item} className="font-bold text-[var(--ods-color-text)]">
{t(`domain_operations_update_${item}`)}
{t(`domain_operations_update_${item}`.toLowerCase())}
</li>
Comment on lines 83 to 86
{t(
`domain_operations_update_nicowner_click_${argumentKey === 'corporationProof' ? 'nicowner' : argumentKey ?? ''
}`,
}`.toLowerCase(),
)}
}}
>
{t(`domain_operations_update_${argumentKey}_click`)}
{t(`domain_operations_update_${argumentKey}_click`.toLowerCase())}
Copilot AI review requested due to automatic review settings July 17, 2026 07:15
louisbensiovh and others added 2 commits July 17, 2026 09:19
ref: #DCE-218

Signed-off-by: Louis BENSI <louis.bensi@corp.ovh.com>
Fresh pnpm installs resolved the transitive @types/react to 19.2.17
while the app runs React 18.2.0, producing 94 TS2786/TS2345 errors
(React 18/19 type mismatch). Pinning @types/react and @types/react-dom
in devDependencies lets manager-pm's critical-deps normalization keep
them on v18, like every other pnpm-managed app.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

<FormField className="w-1/4" invalid={error != null && error.length > 0}>
<FormFieldLabel>
{t(`domain_operations_update_key_${argumentKey}`)}
{t(`domain_operations_update_${argumentKey}`.toLowerCase())}
{fields.map((item) => (
<li key={item} className="font-bold text-[var(--ods-color-text)]">
{t(`domain_operations_update_${item}`)}
{t(`domain_operations_update_${item}`.toLowerCase())}
Comment on lines 83 to 86
{t(
`domain_operations_update_nicowner_click_${argumentKey === 'corporationProof' ? 'nicowner' : argumentKey ?? ''
}`,
}`.toLowerCase(),
)}
Comment on lines +91 to +92
"domain_operations_update_name": "Nom",
"domain_operations_update_uklegalform": "Forme légale",
Copilot AI review requested due to automatic review settings July 17, 2026 07:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

<FormField className="w-1/4" invalid={error != null && error.length > 0}>
<FormFieldLabel>
{t(`domain_operations_update_key_${argumentKey}`)}
{t(`domain_operations_update_${argumentKey}`.toLowerCase())}
{fields.map((item) => (
<li key={item} className="font-bold text-[var(--ods-color-text)]">
{t(`domain_operations_update_${item}`)}
{t(`domain_operations_update_${item}`.toLowerCase())}
Comment on lines 83 to 86
{t(
`domain_operations_update_nicowner_click_${argumentKey === 'corporationProof' ? 'nicowner' : argumentKey ?? ''
}`,
}`.toLowerCase(),
)}
}}
>
{t(`domain_operations_update_${argumentKey}_click`)}
{t(`domain_operations_update_${argumentKey}_click`.toLowerCase())}
Comment on lines +90 to +94
"domain_operations_update_firstname": "Prénom",
"domain_operations_update_name": "Nom",
"domain_operations_update_uklegalform": "Forme légale",
"domain_operations_update_nationalidentificationnumber": "Code fiscal",
"domain_operations_update_identificationnumber": "Numéro d'identification",
Copilot AI review requested due to automatic review settings July 17, 2026 07:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

kb-ovh
kb-ovh previously approved these changes Jul 17, 2026
Signed-off-by: CDS Translator Agent <opensource@ovh.net>
Copilot AI review requested due to automatic review settings July 17, 2026 14:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants